Skip to content

build: activate correct default flags for riscv64 - #65708

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
sxa:rvcompiles
Sep 3, 2026
Merged

build: activate correct default flags for riscv64#65708
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
sxa:rvcompiles

Conversation

@sxa

@sxa sxa commented Sep 1, 2026

Copy link
Copy Markdown
Member

This enables two default flags to control the build on RISC-V.

  1. -march=rv64gc stops the compiler from creating a binary which may have vector (RVV) instructions. The machines that we would use for building on RISC-V tend to have vector instructions and so the compilers on those distributions will be likely to be creating RVV (rv64gc) code by default. This does not impact V8's ability to generate vector instructions at runtime and exploit them on newer machines that have them.
  2. Disable RVV in Highway as that can cause problems as described in the footnotes of Tracking issue for problems on RISC-V Linux platform build#4099 (comment)

Signed-off-by: Stewart X Addison <sxa@ibm.com>
@sxa sxa self-assigned this Sep 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/v8-update

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to Node.js builds or CI infrastructure. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency. labels Sep 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@kxxt

kxxt commented Sep 2, 2026

Copy link
Copy Markdown
Member

2. Disable RVV in Highway as that can cause problems as described in the footnotes of Tracking issue for problems on RISC-V Linux platform build#4099 (comment)

I think this has been fixed in google/highway@4a43b15, though I haven't tested it.

Edit: V8 hasn't bumped highway for a long time so this fix has not reached v8 yet.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@sxa sxa added the commit-queue PRs queued for automated landing through the Commit Queue. label Sep 3, 2026
@sxa
sxa requested a review from richardlau September 3, 2026 10:29
@nodejs-github-bot
nodejs-github-bot merged commit c6ab55c into nodejs:main Sep 3, 2026
87 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in c6ab55c

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to Node.js builds or CI infrastructure. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants